wayland: Only bind to supported gtk-shell versions

The gtk-shell Wayland protocol extension is not meant to be backward
compatible right now, so avoid binding to any version that is not the
one supported.

https://bugzilla.gnome.org/show_bug.cgi?id=745721

build: Fix generating introspection on MacOS

We need to filter out platform-specific sources from the files that we
use with the introspection scanner.

In this specific case I could have moved the gtkclipboard-quartz.c and
the gtkdnd-quartz.c files out of the $(gtk_base_c_sources) variable, but
doing a filter-out on the variable itself is more resilient in case we
eventually add files and we forget about the result.

inspector: Add some margin to css editor

Requested in
https://bugzilla.gnome.org/show_bug.cgi?id=745918

window: reset both press/drag gestures when move/resize drag starts

Now that this is split in two separate gestures, both must be reset
when the WM grabs the pointer. Also, do on resize drags like on move
drags, and claim the gesture before resetting, so the ownership is
properly transferred across any other widgets.

https://bugzilla.gnome.org/show_bug.cgi?id=745969

Updated Swedish translation

Updated Swedish translation

notebook: Fix boolean check

Commit c352093bdeb2be5390607de13eaf7a21e93272ef swapped the condition
when turning it into a return_if_fail().

entrycompletion: Ensure transient_for is set before showing window

gtk_entry_set_completion() (and _gtk_entry_completion_connect underneath)
may happen when the entry has no toplevel yet. This window acts as the
"parent" for subsurface windows on wayland, and is indispensable to create
its backing wl_subsurface, so this resulted in non-visible popups there.
Fix this by ensuring transient_for is set before showing the popup.

https://bugzilla.gnome.org/show_bug.cgi?id=745463

gtknotebook: Fix potential use of uninitialised variables

Found by scan-build.

https://bugzilla.gnome.org/show_bug.cgi?id=712760

gtkprintunixdialog: Fix a potential g_object_unref(NULL) call

The code above checks whether (printer == NULL), so we’d better do so
here as well.

Found by scan-build.

https://bugzilla.gnome.org/show_bug.cgi?id=712760

Revert "csd: Drop the GTK_FRAME_EXTENTS requirement"

This reverts commit fb9a6bb6d8d6b60b25c9b9853decbcea938f7863.

In a recent test, I've found that Xfce and Mate now support
this, so they will not be affected by this requirement. And
adding the check back will solve the 'client-side shadow'
problem in KDE.

stylecontext: Store the right style in the cache

Store the newly computed style, not the old and wrong one.

Fixes HighContrast menuitem color sometimes being black-on-black for
selected items.

gtkclipboard: Fix a potential g_signal_handler_disconnect(NULL) call

Found by scan-build.

https://bugzilla.gnome.org/show_bug.cgi?id=712760

gtkprintcontext: Fix several potential g_object_[un]ref(NULL) calls

The page_setup of a GtkPrintContext or GtkPrintUnixDialog is nullable,
so all reference count changes to it have to be guarded against NULL
values.

Found by scan-build.

https://bugzilla.gnome.org/show_bug.cgi?id=712760

gtktreeselection: Fix a potential use of uninitialised variables

Found by scan-build.

https://bugzilla.gnome.org/show_bug.cgi?id=712760

gtktreeselection: Fix potential NULL pointer dereferences

_gtk_rbtree_first() can potentially return NULL if the RB tree is empty,
which would result in NULL pointer dereferences in the GtkTreeSelection
code. Gracefully handle them.

Found by scan-build.

https://bugzilla.gnome.org/show_bug.cgi?id=712760

gtktextview: Fix a definite use of an uninitialised variable

This seems to have been a typo in the original code, and allowed access
to virtual_cursor_y when it was uninitialised.

Found by scan-build.

https://bugzilla.gnome.org/show_bug.cgi?id=712760

gtknotebook: Remove dead variable assignments

Found by scan-build.

https://bugzilla.gnome.org/show_bug.cgi?id=712760

gtknotebook: Fix a potential NULL pointer dereference

Found by scan-build.

https://bugzilla.gnome.org/show_bug.cgi?id=712760

gtkentry: Return early from gtk_entry_clear() if no icon info exists

This helps scan-build avoid some false positive potential NULL pointer
dereference warnings.

https://bugzilla.gnome.org/show_bug.cgi?id=712760

gtkentry: Remove dead assignments

Found by scan-build.

https://bugzilla.gnome.org/show_bug.cgi?id=712760

gtkentry: Call get_*_size() vfuncs unconditionally

These vfuncs cannot be NULL: implementations are provided by GtkEntry,
and subclasses should not set them to NULL. Instead of conditionalising
the calls to the vfuncs, assert that they’re set and call them
unconditionally.

This prevents the possibility of a subclass setting the vfunc to NULL
and then a gtk_entry_get_*_size() call returning undefined values in its
out variables.

https://bugzilla.gnome.org/show_bug.cgi?id=712760

gtkentry: Fix potential use of uninitialised variables

Found with scan-build.

https://bugzilla.gnome.org/show_bug.cgi?id=712760

gdkwindow: Fix potential NULL pointer dereference

gdk_window_ensure_native() can end up with a NULL parent pointer, which
it passes to find_native_parent_above()…but that expects a non-NULL
parent.

Found with scan-build.

https://bugzilla.gnome.org/show_bug.cgi?id=712760

Prevent empty password request popup for kerberos

    If auth_info_required is only set to "negotiate" the
    request password dialog is shown anyway for each print
    job without any input fields. As the request_password
    dialog only handles username, password and domain.

    https://bugzilla.gnome.org/show_bug.cgi?id=669008

Update hello-world.c to GtkApplication, document

Print dialog: Fix printer icons in HighContrast

We need to make the cell renderer follow the state,
for recoloring to work.

HighContrast: Some formatting fixes

Add a missing word in the docs

Updated Lithuanian translation

docs: Reword the deprecation notice for gtk_widget_override_*

They are a bit terse, and they should point to the preferred way of
overriding the theme, i.e. using a GtkStyleProvider for your
application, and custom style classes.

docs: Clarify replacement for gtk_style_context_get_background_color()

It seems odd that a getter function for the style context state is
replaced by a rendering call, but it makes sense in context.

Add a test for builder-constructed file filters

GtkFileFilter: Set name from builder id

Suggested in https://bugzilla.gnome.org/show_bug.cgi?id=577862
as a way to make file filters constructed with GtkBuilder more
useful.

Updated Kazakh translation

docs: fix reference to GTK+ 2 index of deprecated symbols

Fix undefined reference to get_xkb()

When configuring Gtk+ with --disable-xkb, the build fails because of an
undefined reference to get_xkb().

This patch fixes this issue.

Signed-off-by: Eric Le Bihan <eric.le.bihan.dev@free.fr>
https://bugzilla.gnome.org/show_bug.cgi?id=739070

Modernize the xReply check

Using AC_CHECK_TYPE instead of AC_TRY_COMPILE avoids compiler
warnings from -Werror.

https://bugzilla.gnome.org/show_bug.cgi?id=535929

Avoid calling g_object_ref on NULL

gtk_widget_set_visual() allows NULL for the visual, so be
careful ot to ref it unconditionally.

scrolled window: Fix size requisition with overlay scrolling

When the scrollbars are overlayed, the size requisition of the
scrolled window should not depend on whether the scrollbars are
visible or not. This was not quite the case, because we forgot
one case where scrollbar size was still added to the requisition.

Add an example for accelerators with manual image menuitems

This is somewhat tricky to work out, so put some example
code here for future reference. The tricky part is that
GtkAccelLabel tries to be smart about hiding the accel
if there's not enough space, so we have to make sure to
pack the label with expand=TRUE and set align=GTK_ALIGN_FILL,
or things won't work.

test/gtkgears.c: Fix on C89 Compilers

Make sure that variables are declared at the top of the block.

Break up one of the sincos() calls into individual calls to sin() and cos()
so that we do not have to complicate the initialization of the following
GLfloat array.

Updated Korean translation

gl: Warn the user if they request a GL context version less than 3.2

If the user requests a version less than 3.2 the version is forced to 3.2.
Previous checking code have an inconsistent behavior depending on which
minor version number was specified. This is avoided now by temporarily
converting the major/minor pair into a single integer.

https://bugzilla.gnome.org/show_bug.cgi?id=744288

Update config.h.win32.in for sincos()

There was an added check in configure.ac for sincos() for gtkgears.c, where
that function is actually a GCC- and glibc-specific extension, which is
not available on Windows.  Reflect on this in config.h.win32.in for the
Visual Studio/Windows builds.

MSVC Builds: "Install" .pdb Files

"Install" the .pdb files that were generated along with the build, to make
it easier for people to develop and debug GTK+, especially when used as a
stack, and as the .pdb files are already generated with all builds.

Also make the copy of the .dll, .lib and .exe files more selective so that
we do not accidently copy files that are not meant to be "installed", or
end up making extra copies of plugin DLLs in the wrong places, when the
projects here are included as a part of a grand solution that is used to
build the entire GTK+ stack, for example.

MSVC 2010+ Builds: Fix .pdb Generation

For Visual Studio 2010 and later, the .pdb filename needs to be explicitly
specified so that it will match the filename of the target, if the project
name differs from the filename of the target.  Make sure that is the case
for the 3 DLLs that we build for Visual Studio 2010 and later.

Updated Polish translation

Improve some GtkStock deprecation notes

Point at edit-delete and edit-paste as available icons.

Updated Slovak translation

Updated Slovak translation

window: Add comment documenting the target widget checks during window drag

Was suggested during review of
https://bugzilla.gnome.org/show_bug.cgi?id=745562#c2

Updated French translation

HighContrast: Make sure selections are visible in infobars

Pointed out in https://bugzilla.gnome.org/show_bug.cgi?id=745622

Adwaita: Make sure selections are visible in infobars

Pointed out in https://bugzilla.gnome.org/show_bug.cgi?id=745622

widget: Improve hack to ignore drags from widgets using motion events

Postpone until the last moment whether the target widget still
potentially uses updates from this sequence, or window dragging
actually applies because all gestures on the target went to denied
state.

This fixes window dragging on empty space in a headerbar that is
contained in a paned (as in e.g. gedit).

https://bugzilla.gnome.org/show_bug.cgi?id=745562

Ensure print-related types are registered

These types are not covered by g_test_register_all_types,
and having the types registered when a print dialog is used
helps in cases where GTK+ is dlopened.

https://bugzilla.gnome.org/show_bug.cgi?id=745065

places sidebar: compare bookmarks by index

GtkPlacesSidebar applies a sorting function on
the tree model that does not consider the case
of bookmarks, which are sorted by their indexes.

By adding the bookmarks corner case and comparing
then by their indexes, GtkPlacesSidebar can sort
the bookmarks properly in the order they're saved.

https://bugzilla.gnome.org/show_bug.cgi?id=744589

Check for existence of sincos

It apparently doesn't exist everywhere.

https://bugzilla.gnome.org/show_bug.cgi?id=745578

file chooser: Really stop typeahead search popups

GtkTreeView is a bad widget and constantly resets the search-column
behind our back. We need to re-unset it every time a model is set :-(

Updated Galician translations

make-pot: Document variables

Damn-lies will have to use one of these to override the name
of the generated file. Lets document them all, while I still
remember this.

filechooser: Only do search_start_query() while we are in search mode

This signal can be emitted by GtkSearchEntry after search has been
cancelled, and other operation mode is set. It doesn't make sense to
populate the search model in that state anymore, so just avoid doing it.

https://bugzilla.gnome.org/show_bug.cgi?id=745479

searchenginetracker: Keep a reference on the search engine while querying

The object might be destroyed when mid operation, causing crashes as the
query callback still expects the object pointer to be valid. Also, remove
the gdk_threads_enter/leave pairs, the callback will be executed on the
caller (UI) thread, so this is not necessary.

https://bugzilla.gnome.org/show_bug.cgi?id=745479

filechooser: Disconnect signal handlers from search engine before destroying

The search engine might stay alive longer due to extra temporary refs, so
the signal handlers should be removed for the filechooser to ignore these
properly.

https://bugzilla.gnome.org/show_bug.cgi?id=745479

Make a pot generating command available

Translators don't want to run autogen before generating pot,
so give them a script.

MSVC 2008 Builds: Speed Up Release Builds

Use the /MP compiler option, where the build time for release builds can
be cut down by quite a bit.  This will however cause a brief warning with
debug builds due to the use of /Gm, but the code will otherwise build
normally.  Unlike the Visual Studio 2010+ builds, we can't use /d2Zi+ as
Visual Studio 2008 does not support that, so we can't get a better
debugging experience for release builds here.

Fix "MSVC Builds: Update icon-browser Projects"

The Visual Studio 2008 Project was not updated correctly, so fix that.

tooltip: remove unused variable and related g_object_set

We never use the value of has_tooltip. This is a left-over from the refactoring
of gtk_tooltip_run_requery().

MSVC Builds: Update icon-browser Projects

There is a new source file that needs to be built, so fix the projects...

MSVC Builds: Improve Build Speed and Debugging

Use Multiprocessor compilation which can cut down build times by quite a
bit and use the /d2Zi+ flag to have better debugging info being logged to
the .pdb for release builds.

These are only applicable for Visual Studio 2010/2012 and later.

gtkswitch.c: Fix Build on C89 Compilers

This file now uses round(), which was not available until C99, so include
fallback-c89.c instead of math.h, which includes math.h and does a fallback
implementation of round().

3.15.10

Avoid stray output from configure

The new xgettext check was leaving a messages.po file behind.

Fix 7c4bf742e82d812ecc5b0c3280db86d2689eb093

The objects-finalize test constructs gestures with NULL as
a widget. The recent addition of weak references was not
ready for that.

gears: Update to slightly more modern OpenGL

Instead of using glxgears, which still uses OpenGL 2.1 and the fixed
pipeline, we use a slightly modified es2gears, OpenGL 3.2, and the
programmable pipeline.

file chooser: Avoid animated scrolling for editing

Avoid scrolling animation when scrolling to where we want
to insert the new folder, since adding the entry doesn't
do the right thing if the treeview is still scrolling.

An alternative would be to wait for the animated scrolling
to reach its target before starting the editing, but this
is easier.

https://bugzilla.gnome.org/show_bug.cgi?id=729366

file chooser: Avoid triggering assertions

cairo surfaces are boxed types, so don't try to set them
with g_value_set_object().

file chooser: Avoid warnings from the location column

Avoid criticals that would come out of this code if file is / or NULL.

Updated French translation

x11: Store last axes from device

And use these for the missing axes if the valuator mask is incomplete.
This used to work fine on tablets because the Wacom driver ensures all
valuators are sent, which is not true if using the evdev driver.

https://bugzilla.gnome.org/show_bug.cgi?id=703610

scrolledwindow: Cancel kinetic/overshoot animation on captured scroll events

This ensures the animation is cancelled if the child widget happens to
GDK_EVENT_STOP scroll events.

https://bugzilla.gnome.org/show_bug.cgi?id=745315

Updates

scrolledwindow: Add motion mask to the widget window

This is not specified specifically by the attached controllers, so let
the scrolledwindow set the mask, as motion events with no buttons pressed
are interesting to it.

https://bugzilla.gnome.org/show_bug.cgi?id=745344

Updated Hebrew translation

wayland: Delay wl_subsurface interface creation until the window is shown

It isn't really necessary anytime before, so just make sure it's there only
when the window is visible.

https://bugzilla.gnome.org/show_bug.cgi?id=743427

gtkwindow: Do not show subsurface windows for hidden popover widgets

This is really just necessary when we have a visible widget, otherwise
the window would temporarily linger with odd size and surface.

https://bugzilla.gnome.org/show_bug.cgi?id=743427

wayland: Thaw the clock if we hide a GdkWindow mid-frame

When a window is hidden, its surface and all its roles are destroyed,
if this happens when we already issued a wl_surface_commit and are
awaiting for a frame callback, the clock will remain frozen for the
next time the window is shown.

To avoid this, keep track of the wl_surface_frame() calls issued,
and ensure the clock is thawed after hiding. If we happen to receive
the frame callback, it is just ignored.

https://bugzilla.gnome.org/show_bug.cgi?id=743427

gtkwindow: Move window dragging to a standalone drag gesture

The gesture is hooked to the capture phase, so it works for buttons in
header bars and whatnot. In order to be friendly to the widget it is
capturing events from, an ugly hack is in place to avoid capturing
events when the target widget has a gesture that would consume motion
events.

gtkmenubutton: Popup menu/popover on GtkButton:clicked

This happens on button release, which is more convenient if the gesture
can be consumed by something else (eg. window dragging), and already behaves
correctly wrt cancelled gestures, broken grabs, etc.

This also allows us to unify pointer and keyboard behavior, popping up the
menu widget in a single place.

gesture: On cancel, remove touchpoint before checking the recognized state

Otherwise the touch being removed doesn't account in recognition.

button: Ensure the button is "left" when a gesture is cancelled

This prevents from stale widget states if the gesture happens to be
cancelled mid-press.

eventcontroller: Chain up on constructed

https://bugzilla.gnome.org/show_bug.cgi?id=745225

eventcontroller, widget: Don't crash if destroyed before the other

There are two scenarios. A widget sub-class owns a GtkEventController
and passes itself to it, or a controller owned by something else is
passed a widget.

In the second case, if the widget is destroyed before the controller,
we will have a crash when destructing the controller because we will
be accessing invalid memory. Adding a weak reference on the widget
addresses that problem.

This leads to a crash in the first case. When the widget is getting
destroyed, it will drop the reference to its own controller. The
controller will skip touching the widget because the weak reference
would have turned it to NULL. However, when the widget sub-class chains
up to GtkWidget it will try to free all the controllers in its list.
Unfortunately, all these controllers have already been destroyed. So
we need to guard against this too.

https://bugzilla.gnome.org/show_bug.cgi?id=745225

gtk_label_set_yalign(): Fix xalign/yalign typo in parameter name.

printing: Set printer state correctly

Consider NULL printer state message as empty too.
Printer state IPP_PRINTER_STOPPED can be considered as paused state.

https://bugzilla.gnome.org/show_bug.cgi?id=743323

gtk_misc_set_alignment(): Improve the deprecation documenation.

To link to GtkWidget.halign/valign and to mention
GtkLabel.xalign/yalign.

gdkglcontext-win32: fix variable names in GDK_NOTE

These probably weren't caught because GDK_NOTE was disabled when the
change was tested.

https://bugzilla.gnome.org/show_bug.cgi?id=745400

gdkglcontext-win32: remove unused variables

https://bugzilla.gnome.org/show_bug.cgi?id=745400

Updated Czech translation